-
-
Notifications
You must be signed in to change notification settings - Fork 165
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add LTI configuration info to the LTI course map page. #2586
Add LTI configuration info to the LTI course map page. #2586
Conversation
fd82200
to
083b8d2
Compare
bef0cc6
to
0277062
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do I understand right that this loads a course environment for each course and gets the relevant values, and then displays them?
It shouldn't hold up this PR, but I wondered about displaying more that could be relevant to an admin. Like preferred_source_of_username
and other things. The config chain can be such that these things don't have the site settings for them, and an admin could see some of those things without digging, turning on LTI debugging, etc.
I was thinking eventually more could be added. I want sure what would be beneficial to add though, and didn't want to just add everything. Perhaps a general course environment inspector page might be something to think about adding to the admin course. A page that shows all course environment resolved values for a course. I am not sure how feasible that is though. |
0277062
to
1731d34
Compare
I realized that I chose the things that are shown because they are the distinguishing LTI parameters that are used to determine the correct course to map to when an LTI content generation request is received. I.e., the things that are important for determining the correct course to use in the LMS course map. So only the Note that the However, this could be changed to show all of the parameters listed above as well as perhaps others (like the |
There are two approvals, I'll merge this for now, discussion of showing more stuff could be in a different PR, or wait a bit? |
I beat you to it :) Having other stuff could be a different discussion. Some page like @drgrice1 describes, where you would click a course name and then see all its environment variables might be nice. |
Merging now is fine. More can possibly be added later. It also occurred to me that adding other things like the other LTI parameters for LTI 1.1 and 13 that I listed before may obsure the reason that the parameters that are currently shown are the ones that are shown. That is what I said before, in that they are the important factors for determining the correct course to map to. Also, I want to correct my comment on the LTI 1.1. consumer key. That is that the ConsumerKey for LTI 1.1 will be unset in most cases. The only time it needs to be set is if there are multiple courses from different LMS's that have the same LMS "context" id (not "course" id as I said before) and both use a course on the same webwork2 server. I copied that from the comment in the authen_LTI_1_1.conf file. Apparently when I switched from course id to context id (due to @Alex-Jordan's point that the course id may not be unique even from the same LMS), I forgot to change the comment in the authen_LTI_1_1.conf file. |
I have had this branch sitting here for a while (since March). Time to put it in.
This adds LTI configuration information to the course admin LTI course map page. This information is probably useful when managing the course map.